Skip to content

Add Prettier formatter with shell script support#4

Open
nyatinte wants to merge 5 commits intomainfrom
claude/add-formatter-011CV3PJiKtXe6e7TK66HfS2
Open

Add Prettier formatter with shell script support#4
nyatinte wants to merge 5 commits intomainfrom
claude/add-formatter-011CV3PJiKtXe6e7TK66HfS2

Conversation

@nyatinte
Copy link
Owner

  • Add Prettier and prettier-plugin-sh for unified formatting
  • Configure formatting for JSON, YAML, Markdown, and Shell scripts
  • Enable pre-commit hooks for format checking via lefthook
  • Add npm scripts for formatting and format checking
  • Format existing files with Prettier

- Add Prettier and prettier-plugin-sh for unified formatting
- Configure formatting for JSON, YAML, Markdown, and Shell scripts
- Enable pre-commit hooks for format checking via lefthook
- Add npm scripts for formatting and format checking
- Format existing files with Prettier
- Remove prettier-plugin-sh (deprecated dependency)
- Use shfmt for shell script formatting
- Add shellcheck for shell script linting
- Update lefthook pre-commit hooks with separate checks:
  - Prettier: JSON, YAML, Markdown
  - shfmt: Shell script formatting
  - shellcheck: Shell script linting (with fallback)
- Update npm scripts for new formatter setup
- Add concurrently package for running formatters in parallel
- Update format and format:check scripts to use concurrent execution
- Add individual scripts for each formatter/checker:
  - format:prettier, format:shell
  - check:prettier, check:shfmt, check:shellcheck
- Improve performance by running checks in parallel
- Add lint:prettier, lint:shfmt, lint:shellcheck to package.json
- Update lefthook.yml to call npm run instead of direct commands
- Centralize command definitions in npm scripts for better maintainability
- Replace npx prettier with npm run lint:prettier
- Add .vscode/settings.json with formatOnSave enabled
- Configure Prettier for JSON, YAML, Markdown
- Configure shell-format for shell scripts
- Add .vscode/extensions.json with recommended extensions
- Update README with setup instructions and tool installation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants